Skip to content
This repository has been archived by the owner on Aug 23, 2019. It is now read-only.

Error: Already Piped FIX #249

Merged
merged 1 commit into from
Apr 3, 2018
Merged

Error: Already Piped FIX #249

merged 1 commit into from
Apr 3, 2018

Conversation

harrshasri
Copy link
Contributor

Chore which i think fixes this issue also
https://github.com/libp2p/js-libp2p-switch/issues/235

@harrshasri harrshasri changed the title Uncaught Error: already piped fix Uncaught Error: already piped FIX Apr 3, 2018
@harrshasri harrshasri changed the title Uncaught Error: already piped FIX Error: Already Piped FIX Apr 3, 2018
@harrshasri
Copy link
Contributor Author

@diasdavid Can you accept this fix

Copy link
Member

@daviddias daviddias left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks @harrshasri :)

@harrshasri
Copy link
Contributor Author

@diasdavid Do we need to update this package dependency in js-ipfs v28.2 to v28.3

@daviddias
Copy link
Member

@harrshasri note that secio was published as a patch version. The error you are seeing (already piped) is being fixed by #245

@daviddias daviddias merged commit 7c60361 into libp2p:master Apr 3, 2018
@@ -1,6 +1,6 @@
{
"name": "libp2p-switch",
"version": "0.37.3",
"version": "0.37.4",
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ah crap. Please do not update the version manually, our release process does it.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

will fix in master

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Oh Okay. Sorry. I'm a noob.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

no worries at all. Shame on me for not reviewing it properly :)

@harrshasri harrshasri deleted the patch-1 branch April 4, 2018 00:21
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants